-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ENSO portrait plot #1245
ENSO portrait plot #1245
Conversation
@acordonez could you please help checking if the new notebook in this PR (pcmdi_metrics/enso/lib/summary_plot_lib/enso_portrait_plot.ipynb) working okay on your end? It includes using some new functions newly added via this PR, so will need a manual installation of the code in the PR before running the notebook. |
@zhangshixuan1987 you are more than welcome to take a look the notebook and make suggestions. |
@lee1043 I ran pcmdi_metrics/enso/lib/summary_plot_lib/enso_portrait_plot.ipynb and did not encounter any errors. |
@acordonez thank you for checking it! |
@lee1043 : Hi Jiwoo, The metrics look great to me. I saw that this has been merged into the current code branch, and let me try to also implement it in the e3sm-pmp-zppy workflow, and I will update you on the sample figures. |
@zhangshixuan1987 thank you! The content of the notebook also can be found here: http://pcmdi.github.io/pcmdi_metrics/examples/enso_portrait_plot.html. Until releasing a new version, please sync your PMP main branch to the latest, and do "pip install ." to install it in your local environment. |
No description provided.